Add Patch Task Instance Summary#50526
Conversation
pierrejeambrun
requested review from
bbovenzi,
bugraoz93,
ephraimbuddy,
jason810496,
jscheffl,
rawwar,
ryanahamilton and
shubhamraj-git
as code owners
boring-cyborg
bot
added
area:API
pierrejeambrun
force-pushed
the
patch-task-instance-summary
branch
from
3f37926 to
d31279c
Compare
|
Following discussion with Brent, we should probably also support task group there. |
|
When we will want to incorporate task groups we will encounter the same problem, we will not be able to update the I feel like maybe we update it (change the return type) as a bugfix because we will not be able to handle Note to self: |
|
I'm ok with calling the change a bug fix. We supported this in the old UI but it fell through the cracks in 3.0.0 |
potiuk
added
the
backport-to-v3-1-test
|
#50550 PR for the endpoint spec fix. That can be targeted for |
|
Do not merge, need to be updated once #50550 is merged |
pierrejeambrun
removed
the
backport-to-v3-1-test
pierrejeambrun
force-pushed
the
patch-task-instance-summary
branch
from
d31279c to
b76da2e
Compare
pierrejeambrun
force-pushed
the
patch-task-instance-summary
branch
from
b76da2e to
836cabf
Compare
|
I need to update the PR description before merging |
* Add Patch Task Instance Summary * Test small adjustment
5 participants
related to: #48584
Update the
patch_task_instance_dry_runandpatch_task_instance endpoints to handleti summaries(i.e mapped TI without specifying the map_index => all of them will be affected). Default value is changed toNonebut it will not break because it supersede the-1` for unmapped tasks.